GtkMenuButton: Avoid direct button access

We were getting ourselves in trouble by casting touch events
to GdkEventButton and poking directly at their internals. Instead,
use GdkEvent API to get what we need.

This fixes a crash when using the gear menu in epiphany with
touch. The same crash also occurred in testmenubutton.

gtkoverlay: Fix issues with remove

The iteration now progresses past a delete, so make sure we iterate
safely. Also, don't chain up if we removed a child.

testoverlay: Fix indentation

testoverlay: Add test for overlay z ordering

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

testoverlay: Fix test by using add_pass_through_overlay

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

GtkOverlay: Add support for input pass-through overlays

For these widgets we set pass-through on the child window so that
input over these widgets (that are not on a child input window) goes
to the window below the overlay.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

overlay: Add reorder_overlay()

This allows you to control the z-ordering of overlay children

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

gdk: Add gdk_window_set_pass_through

An pass_through window is something you can draw in but does not
affect event handling. Normally if a window has with no event mask set
for a particular event then input events in it go to its parent window
(X11 semantics), whereas if pass_through is enabled the window below
the window will get the event. The later mode is useful when the
window is partially transparent. Note that an pass-through windows can
have child windows that are not pass-through so they can still get events
on some parts.

Semantically, this behaves the same as an regular window with
gdk_window_set_child_input_shapes() called on it (and re-called any
time a child is changed), but its far more efficient and easy to use.

This allows us to fix the testoverlay input stacking test.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

testoverlay: Add new input stacking test

In this case we have a bunch of interactive main children
of the overlay, and then a centered overlay that contains both
non-interactive (labels) and interactive (entry) widgets.

This shows off a problem where the non-interactive parts (the labels)
steals input from the overlay main children (breaks button click and
hover effects).

https://bugzilla.gnome.org/show_bug.cgi?id=750568

https://bugs.freedesktop.org/show_bug.cgi?id=90917

Updated Spanish translation

Updated Greek translation

stylecontext: Change fallback behavior on state mismatch

For functions that take state flags as an argument we need to special
case the situation where the passed in flags don't match the current
state.

Previously we would create a copy of the style info, change its state
and do the lookup from there.

Now that GtkCssNode has replaced style infos, this doesn't work as well
anymore as copying a GtkCssNode is not possible.
However, unike style infos, GtkCssNodes are instant-apply, so we don't
need to copy anymore, we can just change the state of the node.

This causes some invalidations to be queued, but we can take that
performance hit as this is fallback code.

https://bugzilla.redhat.com/show_bug.cgi?id=1228852

widget-factory: Make the circular button do something

testassistant: Remove padding from the filechooser page

This tests the new assistant api to remove padding from
pages.

GtkAssistant: Add a way to have pages without padding

This can be useful when embedding complex widgets like
a file chooser as a page.

https://bugzilla.gnome.org/show_bug.cgi?id=750631

Add a filechooser-in-assistant example

This is one of the cases where one wants a page without
any padding.

GtkAssistant: Redo page margins

Instead of having padding outside the notebook containing
all pages, put each page in an extra box and add the padding
there. This is in preparation for allowing pages without
padding.

assistant: Drop hardcoded padding

This means the pages themselves will be responsible for setting
suitable padding.

https://bugzilla.gnome.org/show_bug.cgi?id=750631

Fixes to Catalan translation

Only build testfontchooserdialog on linux

We are using this to test pango functionality that
uses fontconfig apis, so this doesn't build on OS X
or Windows anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=750892

color editor: make popups toggleable

The popups on the color editor are somewhat hard to get rid of.
Arrange things so that clicking the same control again dismisses
the current popup.

Don't show a menu on the color editor swatch

There's nothing to customize here, so no need to show a menu.

Add a way to avoid menu on color swatches

Avoid uninitialized variables

clang complains that these may be used uninitialized.

entry: Drop an unused variable

Drop an unused variable

Remove unused variables

Ensure a variable is initialized

clang complained about transient_for being used uninitialized.

Remove an unused variable

broadway: Avoid compiler warnings

clang doesn't like self-assignment.

GtkColorEditor: Remove old todo

Most of this is done, and the rest will not happen because
of this note.

widget-factory: More popover examples

Add an example for text input in a popover. This leads to nested
popovers with touch selection, and does not currently work.

widget-factory: Add horizontally linked entries

It happens in some places, so artists need an example to
make it look good.

Fix 'void function should not return void expression' warning for clang

https://bugzilla.gnome.org/show_bug.cgi?id=750888

Fix return value error in _gtk_text_handle_set_direction

https://bugzilla.gnome.org/show_bug.cgi?id=750888

Redo font map support in GtkFontChooser

We can't add properties to the interface, since it breaks
3rd party implementations of the GtkFontChooser interface.
These exist, for example in gnumeric.

So, instead of a new property, add getter/setter vfuncs.

fontchooser: Don't cause "row-changed" signal in cell data func

The font chooser delays creating the font description from the font face
as long as possible (it's slow). Because we use fixed height mode, we
only have to create font descriptions for rows we are actually going to
show.

This was achieved by looking at the font description column and if it
was NULL, we created a font description and gtk_list_stiore_set() it.
Unfortunately this caused a "row-changed" signal to be emitted and this
emission could happen during the cell data func.
And that caused infinite loops with accessibility when you were unlucky.

This change replaces the NULL font description with an empty one and
instead of setting the correct font description, we
pango_font_description_merge() it in. This way, the list store doesn't
change and no signals are emitted.

https://bugzilla.redhat.com/show_bug.cgi?id=1197267

popover: Avoid hiding modal popovers if another popover is taking focus

Due to popover modality itself, there's quite high chances the popover
stealing focus has been triggered from within, so stay friendly to it.
Hiding the popover here will only hide the grabbing popover too if this
happens.

https://bugzilla.gnome.org/show_bug.cgi?id=750741

window: Avoid rising the window of already shown popovers

This breaks stacking, as popovers are repositioned sort of frequently
(even more after show/hide animations).

https://bugzilla.gnome.org/show_bug.cgi?id=750741

reftests: fix style class syntax in CSS file

Fixes reftests.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

testsuite: add a test for non-square symbolic icons

To verify the previous fix.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

icontheme: don't modify symbolic SVG dimensions when recoloring

When recoloring symbolic SVG, do not modify the original width and
height of the passed-in file; the function later will scale the image
through gdk_pixbuf_new_from_stream_at_scale(), but we should still
use the original size to create the proxy SVG, or the image will
possibly be doubly-resized or blurry.

https://bugzilla.gnome.org/show_bug.cgi?id=750605

recent chooser: Make it expand

It is expected that the list fills the available space.

https://bugzilla.gnome.org/show_bug.cgi?id=750690

entry: Ensure we unset the magnifier widget on ::destroy

Otherwise there's dangling signal handlers which cause warnings
at the time we attempt to disconnect these.

textview: Set handles' mode to "none" if showing the bubble

show_or_hide_handles() tries to disable visibility when the popover is
shown, although it triggers a bit late, and lets the handles flash briefly
if both popover and handles try to show at the same time (eg. when
pressing on the selection of a previously unfocused textview, the handles
were previously hidden, so they try to show again on focus in).

textview: Perform tougher handle visibility checks after hiding the popover

The handles might fall outside the visible area, and shouldn't be shown
then. Just call gtk_text_view_update_handles() which will perform these
checks, and keep the handle conveniently hidden.

This was leading to unexpectedly visible handles (and in the
wrong/previous position, the handle code doesn't relocate the widget
it's about to hide) when "select all" was selected in the popover on
a textview needing scrollbars.
and extending the selection beyond the view above and/or below.

popover: Be lenient wrt visibility of popovers too close to widget borders

The check used to hide the popover if the pointed area fell partly out of
the widget allocation, textviews now can trigger that with text selections
too close to the visible edge, as a small extra area around is now reserved.

The check has been changed to only hide the popover if the pointed area
falls completely outside the widget allocation.

texthandle: Remove knowledge about window borders

We need to expand beyond these if necessary, so don't artificially
limit things here.

window: Allow more space at edges for popovers

We're clamping to the visible window edge, which misplaces text handles.

entry: Set text direction information on text handles

This way the handles point to the right side on the presence of rtl/ltr
text.

textview: set text direction information on text handles

This makes them point to the correct place, given the theme supports
this.

texthandle: Handle ltr/rtl positioning

This setting is per handle, as is dependent on the selected text, rather
than the locale.

Adwaita: support rtl on text handles

Just use the opposite variant for the opposite LTR side.

texthandles: Move start handle to bottom

The asset is going to change to point upwards, so physically place
the handle below the text position.

https://bugzilla.gnome.org/show_bug.cgi?id=750692

Adwaita: update text selection handles

- we'll put both handles to teh bottom
- renamed top, bottom to start and end. will be easier
  for rtl too.

https://bugzilla.gnome.org/show_bug.cgi?id=750692

docs: Fix some minor typos in comments in GtkWidget

settings: Fix the type annotation for color-hash

The (element-type) annotation cannot be used with properties that return
container types like GHashTable; we need to use the (type) annotation.

https://bugzilla.gnome.org/show_bug.cgi?id=750489

builder: Skip introspecting lookup_callback_symbol()

Opaque function pointers as return values are not really usable in high
level languages, even via introspection.

docs: Fix annotation for GtkGLArea::resize signal

We need to document all arguments for the signal, for both gtk-doc and
introspection.

docs: Improve deprecation notices for GtkSettings

The button and menu item icons settings are deprecated; application
developers should control whether or not a widget should show an icon,
using the existing API.

https://bugzilla.gnome.org/show_bug.cgi?id=750718

docs: Improve deprecation notices for GtkMisc

Point to the replacement properties and functions.

https://bugzilla.gnome.org/show_bug.cgi?id=750718

docs: Improve deprecation notice for rules-hint property

Specify that the theme is wholly in control.

https://bugzilla.gnome.org/show_bug.cgi?id=750718

docs: Improve deprecation notices for GtkImageMenuItem properties

Should help porting code away from the deprecated GtkImageMenuItem
class.

https://bugzilla.gnome.org/show_bug.cgi?id=750718

docs: Add deprecation notices for GtkAlignment properties

We want people to port their code away from deprecated classes, and this
helps finding a replacement.

https://bugzilla.gnome.org/show_bug.cgi?id=750718

gtklistboxrow: propagate in the draw vfunc

So objects connected by g_signal_connect_after actually get
the signal.
This was causing an issue in the dnd highlight, since there
a cairo rectangle is draw using g_signal_connect_after on the draw
signal.

entry: Select word on double tap

The same approach as taken for GtkTextView in the previous commit.

text view: Select word on double tap

This is an initial attempt at implementing double-tap selection.

gtk-demo: Make hypertext demo work with touch

Make tapping on the links work.

wayland: Improve guessing of popup placement for detached popup menus

If a menu was not attached to any widget, we try to calculate its
position given where the grabbed pointer is and what window has its
focus. Previously we failed to do so if a "transfer window" was used
for the grab, and this patch adds a code path that, if the menu window
itself didn't have the grab, look for the transfer window and get the
grab device from there.

https://bugzilla.gnome.org/show_bug.cgi?id=748951

wayland: Don't try to guess the popup menu placement when it was set

If a position was already explicitly set, don't try to guess the
position of popup menus by looking at the pointer position, just use
the set coordinates.

https://bugzilla.gnome.org/show_bug.cgi?id=748951

wayland: Position popups relative to the parent surface

According to the xdg-shell protocol specification the (x, y) coordinates
passed when creating a popup surface is relative to top left corner of
the parent surface, but prior to this patch, if the parent surface
was an xdg_surface, we'd position it relative to top left corner of the
window geometry of that xdg_surface.

https://bugzilla.gnome.org/show_bug.cgi?id=749717

GtkTextHandle: Improve handle positioning

When we are close the window edge, we need to shrink the 'invisible
border' around the handle to avoid mispositioning it. A fiddly
calculation, but it works.

GtkTextHandle: drop non-implemented functions

GtkTextHandle: position handles properly

Move the handles so that the asymmetric assets align with the
start/end of the selection.

widget-factory: Improve touch selection example

Make the check buttons not take focus on click, so the popover
stays open.

entry: Make touch selection work with mouse

The buttons on the popover where stealing the focus from the text
view on click, causing the popover to be dismissed before the action
was taken. Fix this by making the buttons not take focus on click.

text view: Make touch selection work with mouse

The buttons on the popover where stealing the focus from the text
view on click, causing the popover to be dismissed before the action
was taken. Fix this by making the buttons not take focus on click.

adwaita: style tweaks to touch selections

- use dark theme assets for the checkboxes and radios
- darken the popover for legibility (white bg/black text is
  really the most problematic background to have).

unfortunatley in terms of SCSS structure this is further digging us into the
hole of specificity. It would be much nicer to set the .osd class on the popover
and have everything just work. I'm sure we'll end up using OSD styled popovers
outside the touch context.

https://bugzilla.gnome.org/show_bug.cgi?id=750396

Update Catalan translation

entry: Avoid criticals

After the recent changes, we could end up calling
gtk_entry_update_handles in cases where the text_handle
has not be created (e.g. when dragging text from an entry).
Avoid that.

widget: Clarify docs on gtk_widget_set_double_buffered()

Also make them more scary so people really really don't use it as a
random knob when trying to make things go fast.

//bugzilla.gnome.org/show_bug.cgi?id=750505

main: Push the current event on the stack before we start needing it

The changes in commit 13e22e20300b7312e52bba7d077fc7e231695fc1 made
_gtk_window_check_handle_wm_event() indirectly depend on
gtk_get_current_event_time() which relies on the current event being
available on the current_events stack.

Since the current event is only pushed on the stack afterwards we get
an invalid timestamp which breaks ewmh window moving.

This fixes the issue by pushing the current event before we start
relying on it being there in gtk_main_do_event() and, as a byproduct,
also fixes a potential memory leak when we have a rewritten event and
return early due to _gtk_window_check_handle_wm_event() being TRUE.

https://bugzilla.gnome.org/show_bug.cgi?id=750384

entry: Always clear handles for mouse interaction

There were some cases where we left a handle visible
during mouse interaction, which is a little weird.

entry: Move touch selection a little further away

Otherwise, it looks cramped, and we end up under the finger.

entry: Hide popover when touching elsewhere

Otherwise, the popover becomes 'sticky' and hard-to-dismiss.

entry: Use the drag-started signal

Use the drag-started signal to differentiate between drags that
move a handle and taps on a handle. Show the touch selection popup
for the latter, but not the former.

entry: Make handles come back reliably

We were setting the text handle mode to NONE too aggressively.

entry: Never show handles and popover at the same time

Hide the handles when the popover appears, and brind them back
when it disappears. This will need revisiting if we start using
the popover for mouse interaction as well, where we may not
want handles to show up.

entry: Shorten touch selection timeout

We don't want the popup to appear spontaneously, so eventually
the timeout may go away altogether. For now, shorten it to 50ms,
to avoid rewriting all the places where the timeout is set or
unset.

text view: Always clear handles for mouse interaction

There were some cases where we left a handle visible
during mouse interaction, which is a little weird.

text view: Move touch selection a little further away

Otherwise, it looks cramped, and we end up under the finger.

text view: Use the drag-started signal

Use the drag-started signal to differentiate between drags that
move a handle and taps on a handle. Show the touch selection popup
for the latter, but not the former.

GtkTextHandle: add a drag-started signal

This will be used to discriminate drags from taps.

text  view: Don't popup after drag selection

Don't show the touch selection popover after a selection drag
ends. Instead, the popover will be triggered by an explicit
touch on the selection.

text view: Never show handles and popover at the same time

Hide the handles when the popover appears, and brind them back
when it disappears. This will need revisiting if we start using
the popover for mouse interaction as well, where we may not
want handles to show up.

text view: Shorten touch selection timeout

We don't want the popover to appear spontaneously, so eventually
the timeout may go away altogether. For now, shorten it to 50ms,
to avoid rewriting all the places where the timeout is set or
unset.

Updated Spanish translation

Updated Spanish translation

widget-factory: Improve context menu example

Show the text formatting in the menu / popover.

widget-factory: Demonstrate custom context menu items

Add bold/italics/underline styling to the context menu of
the 'Lorem ipsum...' text view in page 1. The point is not
to show good UI for this kind of styling, but to demonstrate
custom actions in the context menu / touch selection.